Update for UI sidebar task state badges#882
Merged
oliver-sanders merged 2 commits intocylc:masterfrom Nov 28, 2025
Merged
Conversation
a026b19 to
5e33642
Compare
6 tasks
5e33642 to
8ebca57
Compare
Member
Author
|
Just linkcheck failing |
src/reference/changes.rst
Outdated
Comment on lines
+32
to
+36
| :cylc-flow: `8.6 <https://github.com/cylc/cylc-flow/blob/master/CHANGES.md>`__ | ||
| :cylc-ui: `2.11 <https://github.com/cylc/cylc-ui/blob/master/CHANGES.md>`__ | ||
| :cylc-uiserver: `1.8 <https://github.com/cylc/cylc-uiserver/blob/master/CHANGES.md>`__ | ||
| :cylc-rose: `1.7 <https://github.com/cylc/cylc-rose/blob/master/CHANGES.md>`__ | ||
| :rose: `2.6 <https://github.com/metomi/rose/blob/master/CHANGES.md>`__ |
Member
There was a problem hiding this comment.
Ach, should have been done at release time.
@wxtim, If not already in the release instructions (quite possible), would you mind adding it.
@MetRonnie, Linking to master with no URL fragment is just linking to the changes file, not the specified section within it. Which isn't really much use as new releases are added to the top.
Suggest either pinning to the branch (as done in earlier releases):
Suggested change
| :cylc-flow: `8.6 <https://github.com/cylc/cylc-flow/blob/master/CHANGES.md>`__ | |
| :cylc-ui: `2.11 <https://github.com/cylc/cylc-ui/blob/master/CHANGES.md>`__ | |
| :cylc-uiserver: `1.8 <https://github.com/cylc/cylc-uiserver/blob/master/CHANGES.md>`__ | |
| :cylc-rose: `1.7 <https://github.com/cylc/cylc-rose/blob/master/CHANGES.md>`__ | |
| :rose: `2.6 <https://github.com/metomi/rose/blob/master/CHANGES.md>`__ | |
| :cylc-flow: `8.6 <https://github.com/cylc/cylc-flow/blob/8.6.x/CHANGES.md>`__ | |
| :cylc-ui: `2.11 <https://github.com/cylc/cylc-ui/blob/2.11.x/CHANGES.md>`__ | |
| :cylc-uiserver: `1.8 <https://github.com/cylc/cylc-uiserver/blob/1.8.x/CHANGES.md>`__ | |
| :cylc-rose: `1.7 <https://github.com/cylc/cylc-rose/blob/1.7.x/CHANGES.md>`__ | |
| :rose: `2.6 <https://github.com/metomi/rose/blob/2.6.x/CHANGES.md>`__ |
Or specifying the URL fragment in the URL, though this would require updating.
8ebca57 to
b7203b1
Compare
oliver-sanders
approved these changes
Nov 28, 2025
Member
|
Dammit, wrong branch. |
Member
|
@MetRonnie, can you either restore your branch, or open a new PR on 8.6.x |
Member
Author
|
Sorry just noticed it myself: #889 |
Member
|
Np, I should have spotted it! Will tag the release and deploy the docs once merged. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
For cylc/cylc-ui#2331
Requirements check-list
CONTRIBUTING.mdand added my name as a Code Contributor.